Skip to content

Fix arm pop reg access #2718

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 26, 2025
Merged

Conversation

jiegec
Copy link
Contributor

@jiegec jiegec commented May 24, 2025

Your checklist for this pull request

  • I've documented or updated the documentation of every API function and struct this PR changes.
  • I've added tests that prove my fix is effective or that my feature works (if possible)

Detailed description

Fix the reg access type of arm thumb pop instruction.

Test plan

./suite/cstest/cstest ../tests/details/arm.yaml

Closing issues

closes #2713

jiegec added 2 commits May 24, 2025 15:23
In some environment, the iteration order does not follow the
alphabetical order, so that the patches cannot be applied cleanly. Add
extra sorting to ensure the order.
@kabeor kabeor merged commit 8df252a into capstone-engine:next May 26, 2025
20 checks passed
@jiegec jiegec deleted the fix-arm-pop-reg-access branch May 26, 2025 03:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong register access mode reported in thumb pop instruction
3 participants